Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add integration tests for WG over Shadowsocks #6338

Merged
merged 3 commits into from
Sep 2, 2024

Conversation

dlon
Copy link
Member

@dlon dlon commented Jun 10, 2024

Close DES-1025.


This change is Reviewable

@dlon dlon requested a review from Serock3 June 10, 2024 15:36
Copy link

linear bot commented Jun 10, 2024

@dlon dlon changed the title Add integration tests for Shadowsocks Add integration tests for WG over Shadowsocks Jun 10, 2024
@dlon dlon force-pushed the add-shadowsocks-obfuscation branch 3 times, most recently from 09ccd3f to 03c3d54 Compare July 29, 2024 13:36
@dlon dlon force-pushed the test-add-shadowsocks-integration-tests branch from d121bdb to 412f77d Compare July 30, 2024 09:17
@dlon dlon force-pushed the add-shadowsocks-obfuscation branch from d439f0f to ae7faf4 Compare July 31, 2024 07:50
@dlon dlon force-pushed the test-add-shadowsocks-integration-tests branch from 412f77d to 1f03143 Compare July 31, 2024 08:34
@dlon dlon force-pushed the add-shadowsocks-obfuscation branch from 6419bca to 5ce68d3 Compare August 8, 2024 13:34
@dlon dlon force-pushed the add-shadowsocks-obfuscation branch 5 times, most recently from a3ba162 to 854c9ba Compare August 16, 2024 07:14
Base automatically changed from add-shadowsocks-obfuscation to main August 16, 2024 07:22
@dlon dlon force-pushed the test-add-shadowsocks-integration-tests branch from 1f03143 to 30841b9 Compare August 26, 2024 08:22
@dlon dlon marked this pull request as ready for review August 26, 2024 08:23
@dlon dlon force-pushed the test-add-shadowsocks-integration-tests branch 2 times, most recently from 536bcb8 to 161996d Compare August 26, 2024 13:25
Copy link
Contributor

@Serock3 Serock3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a minor nit comment, otherwise everything is looking good! Would be nice to see that the tests pass before merging though.

@dlon dlon force-pushed the test-add-shadowsocks-integration-tests branch 3 times, most recently from cc404c2 to 70c8183 Compare August 26, 2024 14:38
Serock3
Serock3 previously approved these changes Aug 26, 2024
@dlon dlon force-pushed the test-add-shadowsocks-integration-tests branch from 70c8183 to 58d42be Compare August 28, 2024 12:44
Serock3
Serock3 previously approved these changes Aug 29, 2024
@dlon dlon force-pushed the test-add-shadowsocks-integration-tests branch from 58d42be to afe6b82 Compare August 29, 2024 13:36
@dlon dlon force-pushed the test-add-shadowsocks-integration-tests branch 2 times, most recently from 2ca29ec to 8a27f60 Compare August 29, 2024 15:55
@dlon dlon force-pushed the test-add-shadowsocks-integration-tests branch from 8a27f60 to 338d082 Compare September 2, 2024 10:05
Copy link
Contributor

@Serock3 Serock3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 2 of 7 files at r4, 3 of 3 files at r5, 2 of 2 files at r6, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dlon)

@dlon dlon merged commit 020dc16 into main Sep 2, 2024
55 of 56 checks passed
@dlon dlon deleted the test-add-shadowsocks-integration-tests branch September 2, 2024 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants